Skip to content

Open palette search results in splits - #3595

Draft
brsbl wants to merge 8 commits into
bb/command-palette-useful-defaults-thr_82jtfaq8eqfrom
bb/command-palette-split-opening-thr_82jtfaq8eq
Draft

brsbl wants to merge 8 commits into
bb/command-palette-useful-defaults-thr_82jtfaq8eqfrom
bb/command-palette-split-opening-thr_82jtfaq8eq

Conversation

@brsbl

@brsbl brsbl commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

Human comments

What was wrong

Thread search results could only replace the current view; users could not open a result beside their existing thread.

What changed

  • Open selected results in splits with Command/Control-Enter or the labeled “Open in split” row action.
  • Preserve message-match anchors when opening a split.
  • Group status with project metadata; reserve the right side for the selected-row action, without a footer hint.

How you verified

  • Head 1fdbcae3b: CI blocked by workspace-download timeout; the app-1 shard did not reach tests.
  • Chrome for Testing 153: the selected result opens beside the current thread; the parent replaces the current view.
  • Desktop and phone captures confirm status beside project metadata. Previously reviewed behavior; no additional code review.

Desktop comparison

Fresh pixels from the exact source web app revisions below, using the same fixtures, route, selection and CSS viewport at 2× density. Desktop: 1440×900 CSS / 2880×1800 PNG. Phone: 390×844 CSS / 780×1688 PNG.

State Before — #3370 15e439bec After — 1fdbcae3b
Search threads · before typing middle-desktop-threads top-desktop-threads
Search threads · title, follow-up draft, archived and message results middle-desktop-thread-matches top-desktop-thread-matches
Search threads · message match middle-desktop-message-match top-desktop-message-match
State Before — #3370 — replaces current view After — #3595 — opens beside current view
Command-Enter on the selected thread result middle-desktop-open-split top-desktop-open-split

Phone comparison

State Before — #3370 15e439bec After — 1fdbcae3b
Search threads · before typing middle-mobile-threads top-mobile-threads
Search threads · title, follow-up draft, archived and message results middle-mobile-thread-matches top-mobile-thread-matches
Search threads · message match middle-mobile-message-match top-mobile-message-match

The pencil marks an existing thread with an unsent follow-up, not a first-class saved draft resource. Message screenshots verify result rendering, not message-anchor landing. This PR remains draft while exact-head CI is incomplete.

BB-Thread-ID: thr_82jtfaq8eq

AGENT GENERATED

@brsbl
brsbl added this pull request to stack #3383 September 12, 2026 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant